-
Notifications
You must be signed in to change notification settings - Fork 1.5k
[SAASINT-4379] DDS: Jamf Pro v1.0.0 #20768
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SAASINT-4379] DDS: Jamf Pro v1.0.0 #20768
Conversation
Created an Editorial Review card for the Docs team: DOCS-11627 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of small copyedits here and a potential suggestion for the Logs table - thank you!
Review from guttymora is dismissed. Related teams and files:
- saas-integrations
- .github/CODEOWNERS
- .github/workflows/config/labeler.yml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Content LGTM, thanks @shubhamvekariya-crest for the PR and reviewing changes.
Review from jeff-morgan-dd is dismissed. Related teams and files:
- documentation
- jamf_pro/assets/dashboards/jamf_pro_computer_insights.json
- jamf_pro/assets/dashboards/jamf_pro_mobile_insights.json
- jamf_pro/assets/dashboards/jamf_pro_system_server_and_miscellaneous_events.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am approving since the logos have been updated, thanks a lot! If the partner is happy with it feel free to merge.
* Add jamf pro integration * Resolve CI failures * Added dashboard screenshots * Added integration svg * Address review comments * Resolve CI failures * Update logos --------- Co-authored-by: jeff-morgan-dd <[email protected]> Co-authored-by: Gustavo Mora <[email protected]> 8bd3485
What does this PR do?
This is a initial release PR of Jamf Pro integration including all the required assets i.e. Pipeline, Dashboards
Integration Logo Source: https://resources.jamf.com/images/logos/Jamf-Pro-color.png
Motivation
ddsource
query parameter set tojamf-pro
.Review checklist (to be filled by reviewers)
qa/skip-qa
label if the PR doesn't need to be tested during QA.backport/<branch-name>
label to the PR and it will automatically open a backport PR once this one is merged